gdk: Remove a redundant check

We already know desktop_notification_id is not NULL.

Pointed out in https://www.viva64.com/en/b/0793/

messagedialog: Don't initialize twice

We don't need to set these fields more than once.

Pointed out in https://www.viva64.com/en/b/0793/

composetable: Remove a redundant check

We already know seq_index is not NULL here.

Pointed out in https://www.viva64.com/en/b/0793/

x11: A case of argument order confusion

translate_keysym was expecting its arguments the
other way around.

Pointed out in https://www.viva64.com/en/b/0793/

textbtree: Avoid line vs char count confusion

The post_insert_fixup helper function was confused about
its argument order.

Pointed out in https://www.viva64.com/en/b/0793/

listbase: Don't specify the same thing twice

We only need to set EXPLICIT_NOTIFY once.

Pointed out in https://www.viva64.com/en/b/0793/

css: Fix border value parsing

This function was not resetting computed as it meant
to because the last loop was never executed.

Pointed out in https://www.viva64.com/en/b/0793/

vulkan: Fix image uploading by regions

This code did not make sense; it was incrementing
the wrong variable.

Pointed out in https://www.viva64.com/en/b/0793/

Merge branch 'wip/jimmac/colored-list-image-buttons' into 'master'

Adwaita: allow suggested and destructive action buttons in lists

Closes #3643

See merge request GNOME/gtk!3153

Merge branch 'ebassi/for-master' into 'master'

Graphene is a dependency of Gsk, not Gdk

See merge request GNOME/gtk!3149

Adwaita: allow suggested and destructive action buttons in lists

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/3643

Merge branch 'x11-dnd-fix' into 'master'

x11: Handle X-specific targets in drops

Closes #3642

See merge request GNOME/gtk!3151

x11: Handle X-specific targets in drops

This code is very similar to the handling for these
targets in the clipboard case.

Fixes: #3642

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!3150

Graphene is a dependency of Gsk, not Gdk

So let's put it in the right place when building the introspection data.

Link the data url tests statically

Thats the cleaner way to test internal apis.

Add tests for css value transitions

Just a few cases for now, enough to test a recently
fixed regression with shadow transitions.

Merge branch 'uac.meson.master' into 'master'

gtk4-update-icon-cache: Avoid UAC on 32-bit Windows

Closes #3632

See merge request GNOME/gtk!3141

gtk4-update-icon-cache: Avoid UAC on 32-bit Windows

As the program executable name has 'update' in its filename,
gtk4-update-icon-cache.exe is considered to be an installer program on 32-bit
Windows [1], which will cause the program to fail to run unless it is running
with elevated privileges (i.e. UAC).

Avoid this situation by embedding a manifest file into the final executable
that tells Windows that this is not a program that requires elevation.

Fixes issue #3632.

[1]: https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-vista/cc709628(v=ws.10)?redirectedfrom=MSDN,
     under section "Installer Detection  Technology"

Merge branch 'matthiasc/for-master' into 'master'

css: Allow transitioning different-size shadows

See merge request GNOME/gtk!3145

css: Allow transitioning different-size shadows

The code handles it just fine. The length check was
an erronous addition.

Merge branch 'im-context-work' into 'master'

Some im context work

Closes #1004, #186, and #3521

See merge request GNOME/gtk!3143

composetable: Parse hex escapes too

This was a small omission from the Compose file
syntax that doesn't cost us much to support.

Add a test for this syntax too.

Fixes: #1004

Add tests for string values

Add a test that checks we parse values with
multiple characters correctly.

composetable: Support string values in the cache

Change the cache format to include the character
data that we need to hold string values in the table.

composetable: Don't use GSlice for big blobs

This just doesn't make sense. This will use malloc
anyway, so just call malloc directly.

composetable: Keep multi-char values

Keep string values in the table, and return them
from the check function. This commit temporarily
disables the table caching, since the cache format
does not handle string values yet.

Fixes: #186

Merge branch 'wip/silence-bounds-warnings' into 'master'

gdk/toplevelsize: Remove warnings about exceeding bounds

Closes #3035

See merge request GNOME/gtk!3142

composetable: Parse multi-char values

Rewrite the value parsing function to accept strings
that hold more than a single Unicode character.

composetable: Another step towards multi-char values

Change the parser data structures to hold a string, rather
than a gunichar. We still only put a single Unicode character
into it, currently.

composetable: Prepare for multi character values

Make it possible for gtk_compose_table_check to return
a string instead of just a single Unicode character.
Currently, we only ever return strings holding a single
character, still.

imcontext: Prepare for multi-char values

Reshuffle things so we can easily handle values
that are strings instead of just single Unicode
characters.

composetable: Check algorithmic matching

Just some spot checks, enough to verify the
fix in the previous commit.

composetable: Fix algorithmic matching

The code wasn't paying attention to (lack of) nul-termination
in one place, causing it to not match when it should.

composetable: Add tests for compact table matching

Not very exhaustive, just some spot checks.

imcontext: Code cleanup

Get rid of auxiliary check_table function.

imcontext: Move code around

Move all the checking code to gtkcomposetable.c, and
add api that we can use in tests.

composetable: Add tests for matching

This tests the api we use to match key sequences
against compose tables.

imcontext: Use gtk_compose_table_check

Use the just-introduced api.

composetable: Add api to check tables

This copies the check_table code from gtkimcontextsimple.c,
in order to have an api for tests.

Add tests for GtkComposeTable

Add some tests for the code that parses Compose files.

This tests the fix in the previous commit.

composetable: Drop table debug code

This is better off in the tests that we are going to add.

composetable: Parser fixes

We were not handling octal escapes right.

imcontext: Drop GTK_MAX_COMPOSE_LEN

Drop GTK_MAX_COMPOSE_LEN from docs. It is no longer
used by GTK at all.  We leave the define in place
for now, to avoid breaking 3rd party code that might
use it.

composetable: Warn when ignoring things

We should at least give a hint that we've seen the line,
otherwise people will wonder why nothing happened.

composetable: parse long sequences

Allow compose sequences of up to 20 code points.

Fixes: #3521

composetable: Fix an off-by-one

Fix an off-by-one in the code parsing octal escapes
in compose files.

imcontext: Stop using GTK_MAX_COMPOSE_LEN

Allocate the compose_buffer, and resize it when needed
to match the tables we use.

imcontext: Code cleanup

Use g_clear_pointer instead of opencoding it in
multiple places.

gdk/toplevelsize: Remove warnings about exceeding bounds

Sometimes the size will exceed the minimum bounds. For example crazy
applications like the widget factory that contains the world, or when a
user interactively resizes a window to be larger than the monitor the
window is on is.

The former is questionable, but the latter is not, and from here we
can't really see the difference, so just stop complaining.

Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/3035

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

Closes #2319

See merge request GNOME/gtk!3140

imcontext: Allow sequences of length GTK_MAX_COMPOSE_LEN

There was an off-by-one error, making us reject sequences
of this length. But the rest of the code handles them
just fine.

Fixes: #2319

imcontext: Improve an error message

This error message was misleading, as pointed out
by Ralf Jung.

imcontext: Add a precondition check

imcontext: Show preedit for compose sequences

Show the sequences as they are entered, using ⎄ for
the compose key, to match what IBus does nowadays.
Also handle backspace to allow corrections.

imcontext: Update our check for dead keys

A bunch of keysyms for dead keys have been added since this
code was last touched. Update the check to cover the full
range from dead_grave to dead_greek.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!3139

Merge branch 'wip/exalm/consumes-motion' into 'master'

Drag fixes and cleanups

Closes #3513

See merge request GNOME/gtk!3001

Remove a forgotten file

The example series only has 9 steps now. Remove remnants
of step 10.

nativedialog: Add more docs

Emphasize that native dialogs aren't widgets, and are not
kept alive by GTK.

Merge branch 'tool-static' into 'master'

tools: Don't static and dynamic link on libgtk4

See merge request GNOME/gtk!3138

Merge branch 'wip/exalm/activate' into 'master'

listitemwidget: Activate on release instead of press

Closes #3345

See merge request GNOME/gtk!3008

tools: Don't static and dynamic link on libgtk4

4.1.0

tests: Disable the textview-margins reftest

It is too flaky to be useful.

node editor: Add a dark mode toggle

This is useful to see light rendering clearly.

Update Brazilian Portuguese translation

(cherry picked from commit 056c3e11a148ff72638afac0e4e9511a11d9987b)

Merge branch 'mcclurgm-master-patch-63249' into 'master'

Document nullability of gtk_list_box_get_selected_row

See merge request GNOME/gtk!3137

Update Brazilian Portuguese translation

(cherry picked from commit f224c8fab9c1d26e5a09e581b1de9d13fead6d74)

Document nullability of gtk_list_box_get_selected_row

Update POTFILES.in

Merge branch 'fix-unfocus-on-unmap' into 'master'

Fix unsetting focus

Closes #3623

See merge request GNOME/gtk!3136

Fix unsetting focus

Make _gtk_window_unset_focus_and_default queue the changes
for after the next draw. This achieves two things: first,
it avoids invalidating css at the wrong time (e.g. when
setting child-visible during size-allocation), and second,
it defers the focus change until after the widget is
hidden, so that moving the focus has the desired effect
of picking a different, visible widget.

Fixes: #3623

Merge branch 'msvc.nounistd' into 'master'

testsuite/testutils.c: Fix build on Visual Studio

See merge request GNOME/gtk!3131

Merge branch 'gst-vaapi-fix' into 'master'

gtkgstsink: Sync texture before handing it to GDK

See merge request GNOME/gtk!3114

Merge branch 'ci-gstreamer-deps' into 'master'

Ci gstreamer deps

See merge request GNOME/gtk!3133

Merge branch 'ebassi/for-master' into 'master'

Ebassi/for master

See merge request GNOME/gtk!3134

gtkgstsink: Sync texture before handing it to GDK

We need to synchronize when moving the texture between contexts, or we
get glitches with VA-API decoding.

ci: Add mesa-libGLES-devel

Try again, since libglvnd-devel is not the right choice.

build: Disable subproject Cairo tests

There's really no point in running them.

docs: Annotate XML fragments as such

This way we can get syntax highlighting.

docs: Escape bare tags

Otherwise tools processing the description of GtkWidget will be *very*
confused.

docs: Use the appropriate syntax for code block language

docs: Remove stray code block end marker

Merge branch 'gtk-surface-release-4' into 'master'

wayland: Signal gtk-shell surface destruction to the server

See merge request GNOME/gtk!3129

ci: Use v26 of the Fedora image

This includes libglvnd-devel and should fix using the
gstreamer gl support.

ci: Add libglvnd-devel to images

gstreamers gl support included headers from this
package without depending on it. Work around this
packaging error by explicitly adding the needed
dependencies.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

Closes #3615

See merge request GNOME/gtk!3132

testsuite/testutils.c: Fix build on Visual Studio

Visual Studio does not come with unistd.h, but Windows do have write() and
close() in io.h, so include io.h instead of unistd.h on Windows.

For MinGW, unistd.h in turn includes io.h.

gl renderer: Fix viewport computation when rendering offscreen

Fixes #3615

showrendernode: Monitor input file

node editor: Show some default node data

Show case the icon and the render node format this way.

shader builder: Improve error output

node editor: Make help textview monospace

Otherwise the nice markdown tables don't line up.

windowhandle: Use drag threshold instead of double click threshold

Now that we have gtk_drag_check_threshold_double(), be consistent with
other draggable widgets and make sure we don't take over a drag before a
child does.

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/3513

dragsource: Use double coordinates for checking drag threshold

If multiple nested widgets have drag sources on them, both using bubble
phase, we need to reliably pick the inner one. Both of them will try to
start dragging, and we need to make sure there are no situations where the
outer widget starts drag earlier and cancels the inner one.

Currently, this can easily happen via integer rounding: start and current
coordinates passed into gtk_drag_check_threshold() are initially doubles
(other than in GtkNotebook and GtkIconView), and are casted to ints. Then
those rounded values are used to calculate deltas to compare to the drag
threshold, losing quite a lot of precision along the way, and often
resulting in the outer widget getting larger deltas.

To avoid it, just don't round it. Introduce a variant of the function that
operates on doubles: gtk_drag_check_threshold_double() and use it instead
of the original everywhere.

entry: Fix drag threshold check

It was passing offsets as current oordinates.

windowhandle: Don't drag on capture phase

This was needed to work around widgets claiming event sequences on press,
by ignoring them and starting the drag anyway unless they have certain
event controllers on them.

The most visible offender was GtkButton, but since the last commit it
doesn't claim the sequence anymore and we can remove the hack.

checkbutton: Claim sequence on release instead of press

Make it possible to drag windows from check buttons in future.

button: Stop claiming event sequence on press

Currently GtkButton claims the sequence both on press and on release. Stop
claiming it on press and only do it on release, allowing drags to start
from it.

This will allow to remove a hack from GtkWindowHandle.